Skip to content

fix: resolve discriminator defaultMapping into components schemas when bundling#2736

Draft
tatomyr wants to merge 2 commits intomainfrom
fix/resolve-defaultMapping-when-bundling
Draft

fix: resolve discriminator defaultMapping into components schemas when bundling#2736
tatomyr wants to merge 2 commits intomainfrom
fix/resolve-defaultMapping-when-bundling

Conversation

@tatomyr
Copy link
Copy Markdown
Collaborator

@tatomyr tatomyr commented Apr 10, 2026

What/Why/How?

Fixed an issue where discriminator defaultMapping was not resolved when bundling.

Reference

Testing

Screenshots (optional)

Check yourself

  • This PR follows the contributing guide
  • All new/updated code is covered by tests
  • Core code changed? - Tested with other Redocly products (internal contributions only)
  • New package installed? - Tested in different environments (browser/node)
  • Documentation update has been considered

Security

  • The security impact of the change has been considered
  • Code follows company security practices and guidelines

@tatomyr tatomyr self-assigned this Apr 10, 2026
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 10, 2026

🦋 Changeset detected

Latest commit: eed6558

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@redocly/openapi-core Patch
@redocly/cli Patch
@redocly/respect-core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 79.91% (🎯 79%) 6730 / 8421
🔵 Statements 79.35% (🎯 79%) 6969 / 8782
🔵 Functions 83.03% (🎯 82%) 1366 / 1645
🔵 Branches 71.56% (🎯 71%) 4570 / 6386
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
packages/core/src/bundle/bundle-visitor.ts 66.4% 61.76% 100% 66.14% 29, 33-41, 48-56, 63, 72, 80, 85-106, 171-183, 200-201, 213-219, 227-228, 265
Generated in workflow #9398 for commit eed6558 by the Vitest Coverage Report Action

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026

CLI Version Mean Time ± Std Dev (s) Relative Performance (Lower is Faster)
cli-latest 3.512s ± 0.024s ▓▓▓ 1.09x
cli-next 3.213s ± 0.018s ▓ 1.00x (Fastest)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant